Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore!: remove leftovers from when virtuals were symlinks #2524

Merged
merged 1 commit into from
Mar 1, 2021

Conversation

merceyz
Copy link
Member

@merceyz merceyz commented Feb 26, 2021

What's the problem this PR addresses?

There are leftovers in the codebase from the time when virtuals (peer dependencies) were symlinks

Closes #2003

How did you fix it?

Removed them

Checklist

  • I have read the Contributing Guide.
  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@merceyz merceyz requested a review from arcanis as a code owner February 26, 2021 15:03
@merceyz merceyz force-pushed the merceyz/virtual-symlink branch from bd1ff69 to 7b91207 Compare February 26, 2021 18:28
@clemyan
Copy link
Member

clemyan commented Feb 27, 2021

Can virtualFolder also be removed? I think it exists because it gave user control over where the symlinks are created, so it no longer have use cases, On the other hand, its existence can cause problems. (see #1851 (comment))

@arcanis arcanis merged commit 215d1aa into master Mar 1, 2021
@arcanis arcanis deleted the merceyz/virtual-symlink branch March 1, 2021 09:19
@arcanis
Copy link
Member

arcanis commented Mar 1, 2021

Can virtualFolder also be removed? I think it exists because it gave user control over where the symlinks are created, so it no longer have use cases, On the other hand, its existence can cause problems. (see #1851 (comment))

Removing it would require to hardcode the path, which we try to avoid in general. Modifying this setting doesn't really make sense, but this way the exact path is stored along with all others, and can be easily accessed by any plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants